Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
TL;DR
This is the first part about the Customer Price List adding the Customer Price List CRUD screens and the sales invoice line calculation. It is missing:
Proof
and also see
sales.invoice.line.service.spec.ts
.Merge request checklist
Please check if your merge request fulfills the following requirements:
yarn build
) was run locally and any changes were pushed for both API and clientsyarn api:dev
) runs locally and any fixes were made for failurescd clients/admin && yarn start
) runs locally and any fixes were made for failuresMerge request type
Before this merge request
You had to calculate the invoice line price manually.
How you fixed what was wrong
Changed the sales invoice line calculation to use the price list if available.